Organize the code a bit.

Darren Cauthon vor 9 Jahren
Ursprung
Commit
8c73d094ce
1 geänderte Dateien mit 4 neuen Zeilen und 1 gelöschten Zeilen
  1. 4 1
      app/models/agents/webhook_agent.rb

+ 4 - 1
app/models/agents/webhook_agent.rb

@@ -38,10 +38,13 @@ module Agents
38 38
     end
39 39
 
40 40
     def receive_web_request(params, method, format)
41
+      # check the secret
41 42
       secret = params.delete('secret')
43
+      return ["Not Authorized", 401] unless secret == interpolated['secret']
44
+
45
+      #check the verbs
42 46
       verbs = (options['verbs'] ? options['verbs'] : 'post').split(/[,;]/).map { |x| x.strip.downcase }
43 47
       return ["Please use #{verbs.join('/').upcase} requests only", 401] unless verbs.include?(method)
44
-      return ["Not Authorized", 401] unless secret == interpolated['secret']
45 48
 
46 49
       [payload_for(params)].flatten.each do |payload|
47 50
         create_event(payload: payload)